Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding largescale ASR model for speechbrain #49

Merged
merged 4 commits into from
Feb 27, 2025

Conversation

TParcollet
Copy link
Contributor

@TParcollet TParcollet commented Feb 6, 2025

Hello there,

This is adding the scripts and modifications to run the evaluation of https://huggingface.co/speechbrain/asr-conformer-largescaleasr . I don't have an A100 available right now ... AND some of the datasets should not be used on company clusters due to license. Any chance that you could run the evaluation?

Thanks!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since you are using same model but different settings, I don;t think this would play well when HF outputs results as the files are saved based on model_name and eval data. Pls run on your machine to verify this. Otherwise rest all LGTM

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be honest, i've played a bit with a local a100, and the decoding seems to slow compared to what I would have expected (what we see within SB). I'll investigate a bit.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes please don't run any test - there is something fishy. I'll investigate.

@TParcollet
Copy link
Contributor Author

all fixed @Vaibhavs10

@Vaibhavs10 Vaibhavs10 merged commit e7fba76 into huggingface:main Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants